Skip to content

Fix LLM export on in-memory filesystems#3656

Merged
theletterf merged 1 commit into
mainfrom
fix/llm-exporter-in-memory
Jul 16, 2026
Merged

Fix LLM export on in-memory filesystems#3656
theletterf merged 1 commit into
mainfrom
fix/llm-exporter-in-memory

Conversation

@theletterf

Copy link
Copy Markdown
Member

Summary

  • create llm.zip through the exporter-provided IFileSystem instead of physical ZipFile APIs
  • enumerate and copy Markdown files through IFileSystem so docs-builder serve works with MockFileSystem output
  • create the output directory when the exporter is finalized without prior page output
  • add a regression test that builds and inspects the archive entirely in memory

Validation

  • ./build.sh lint
  • dotnet test tests/Elastic.Markdown.Tests/ --no-restore --disable-build-servers -m:1

All 1,895 Elastic.Markdown tests pass.

@theletterf theletterf requested a review from a team as a code owner July 16, 2026 09:57
@theletterf theletterf requested a review from technige July 16, 2026 09:57
@theletterf theletterf self-assigned this Jul 16, 2026
@theletterf theletterf added the fix label Jul 16, 2026
@theletterf

Copy link
Copy Markdown
Member Author

+CC @Mpdreamz I was getting errors related to llms.zip when doing a local preview.

@theletterf theletterf merged commit 5769eb4 into main Jul 16, 2026
24 of 25 checks passed
@theletterf theletterf deleted the fix/llm-exporter-in-memory branch July 16, 2026 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants